Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Long term updates #46

Merged
merged 13 commits into from
Jun 2, 2017
Merged

Long term updates #46

merged 13 commits into from
Jun 2, 2017

Conversation

danirod
Copy link
Member

@danirod danirod commented Jun 2, 2017

A few updates more complex than running gem update.

Update Ruby versions

  • Update Ruby versions
    Should keep the Ruby version running in the server up to date in order to be protected against possible threats. Ruby 2.3.3 is not the latest version. Test compatibility with Ruby 2.3.4 and patch properly if it fails. Start testing for Ruby 2.4.0. I tried this in the past and it fails with Ruby 5.0. Let's hope for a fix as well.

Update to Rails 5.1

  • Update to Rails 5.1
    5.0 will continue receiving security updates, but updating to Ruby 5.1 soon is a better idea.

danirod added 13 commits June 2, 2017 18:34
Ruby 2.3.3 is not the latest version in 2.3 series. Test that the
project works in Ruby 2.3.4 and patch bugs if it doesn't. Plus, start
testing compatibility with Ruby 2.4 series.
The file new_framework_defaults is dropped now since all the defaults
have been merged and unit tests still validate.
Isn't it always a joy running app:update and letting Rails ask you if
you want to remove all your configuration? I have learnt now that the
best way for dealing with this is to just replace every file when
app:update asks so, but then spend time using git add -p.
This seems to be something new on Rails 5.1.
App is fully ported to 5.1.
@danirod danirod merged commit 415b608 into master Jun 2, 2017
@danirod danirod deleted the long-term-updates branch June 2, 2017 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant